Skip to content

fix: Sonarqube reliability and maintainability issues.#862

Merged
pvital merged 2 commits intomainfrom
sonarqube_reliability_fixes
Apr 9, 2026
Merged

fix: Sonarqube reliability and maintainability issues.#862
pvital merged 2 commits intomainfrom
sonarqube_reliability_fixes

Conversation

@pvital
Copy link
Copy Markdown
Member

@pvital pvital commented Apr 7, 2026

This PR fixes a few SonarQube reliability and maintainability issues.

  • reliability:
    • Fixed in src/instana/instrumentation/aws/boto3.py:
      • Low reliability issue Function parameters initial values should not be ignored.
    • Fixed in src/instana/instrumentation/tornado/client.py:
      • Medium reliability issue All "except" blocks should be able to catch exceptions.
  • maintainability
    • Fixed several files for the issue Replace the unused local variable "parent_span" with "_".

pvital added 2 commits April 7, 2026 23:08
- Fixed in `src/instana/instrumentation/aws/boto3.py`:
    - Low reliability issue **Function parameters initial values should
not be ignored.**
- Fixed in `src/instana/instrumentation/tornado/client.py`:
    - Medium reliability issue **All "except" blocks should be able to
catch exceptions.**

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Fixed several files for the issue **Replace the unused local variable "parent_span" with "_".**

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital self-assigned this Apr 7, 2026
@pvital pvital added the fix label Apr 7, 2026
@pvital pvital marked this pull request as ready for review April 7, 2026 21:59
@pvital pvital requested a review from a team as a code owner April 7, 2026 21:59
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
7.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Copy Markdown
Contributor

@GSVarsha GSVarsha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@pvital pvital merged commit 6e9098e into main Apr 9, 2026
19 of 20 checks passed
@pvital pvital deleted the sonarqube_reliability_fixes branch April 9, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants